TF-PSA-Crypto Doxygen Adaptations #115
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Harry-Ramsey
force-pushed
the
tf-psa-crypto-doxygen
branch
2 times, most recently
from
3a2fde9 to
b22c1b6
Compare
3 tasks
Harry-Ramsey
force-pushed
the
tf-psa-crypto-doxygen
branch
2 times, most recently
from
fe80451 to
41789ad
Compare
ronald-cron-arm
requested changes
Jan 17, 2025
Harry-Ramsey
force-pushed
the
tf-psa-crypto-doxygen
branch
from
41789ad to
a86eefa
Compare
Harry-Ramsey
force-pushed
the
tf-psa-crypto-doxygen
branch
from
59e9177 to
f16b29c
Compare
ronald-cron-arm
requested changes
Feb 4, 2025
scripts/apidoc_full.sh
Outdated
Comment on lines
35
to
41
| elif in_tf_psa_crypto_repo; then | ||
| cmake -DCMAKE_BUILD_TYPE:String=Check -DGEN_FILES=ON . | ||
| make tfpsacrypto-apidoc | ||
| mv $CONFIG_BAK $CONFIG_H | ||
| fi |
There was a problem hiding this comment.
Suggested change
| elif in_tf_psa_crypto_repo; then | |
| cmake -DCMAKE_BUILD_TYPE:String=Check -DGEN_FILES=ON . | |
| make tfpsacrypto-apidoc | |
| mv $CONFIG_BAK $CONFIG_H | |
| fi | |
| elif in_tf_psa_crypto_repo; then | |
| scripts/config.py realfull | |
| cmake -DCMAKE_BUILD_TYPE:String=Check -DGEN_FILES=ON . | |
| make tfpsacrypto-apidoc | |
| mv $CONFIG_BAK $CONFIG_H | |
| fi |
and add realfull adapter in config.py in TF-PSA-Crypto the same way it is done in config.py in mbedtls.
scripts/check-doxy-blocks.pl
Outdated
| @@ -16,7 +16,8 @@ | |||
| use File::Basename; | |||
|
|
|||
| # C/header files in the following directories will be checked | |||
| my @directories = qw(include/mbedtls library doxygen/input); | |||
| my @mbedtls_directories = qw(include/mbedtls library doxygen/input); | |||
| my @tf_psa_crypto_directories = qw(include/psa drivers core doxygen/input); | |||
There was a problem hiding this comment.
Suggested change
| my @tf_psa_crypto_directories = qw(include/psa drivers core doxygen/input); | |
| my @tf_psa_crypto_directories = qw(include/psa include/tf-psa-crypto | |
| drivers/builtin/include/mbedtls | |
| drivers/builtin/src core doxygen/input); |
Harry-Ramsey
force-pushed
the
tf-psa-crypto-doxygen
branch
from
f16b29c to
171fd57
Compare
This commit adapts check-doxy-blocks to run for TF-PSA-Crypto. Signed-off-by: Harry Ramsey <[email protected]>
This commit adapts the scripts apidoc_full.sh and doxygen.sh to run for TF-PSA-Crypto out of source builds. Signed-off-by: Harry Ramsey <[email protected]>
Harry-Ramsey
force-pushed
the
tf-psa-crypto-doxygen
branch
from
171fd57 to
a256c44
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Adapt doxygen scripts to work for TF-PSA-Crypto.
PR checklist
Please add the numbers (or links) of the associated pull requests for consuming branches. You can omit branches where this pull request is not needed.
Notes for the submitter
Please refer to the contributing guidelines, especially the
checklist for PR contributors.
Help make review efficient: